gtktreeview: Transform cell area to widget coordinates

GtkCellArea uses event coordinates (thus in treeview relative
coordinates), but calculations used to happen in bin window coords.
We can just offset the cell area by the bin window, fixes cell
renderer activation and edition.

treeviewcolumn: Don't swallow motion/crossing events

If the column is not clickable, it may make some sense to stop
event propagation here for button events. However motion events
should be left alone.

Fixes treeview column resize pointer cursors, since that's
implemented up the bubbling phase in the treeview.

treeview: Transform motion event coords to "bin window" coords

The operations rely there on bin window relative coordinates, but we
are receiving GtkTreeView relative coordinates there. Fixes clicking
on treeview expanders, which was offset by visible headers.

modelbutton: set_relative_to(NULL) the popover in dispose

We do set_relative_to(menu_button) when creating it so we should do this
here (or nothing).

treeviewcolumn: Don't leak button

We g_object_ref_sink the button in _create_button, so we need to unref
it ourselves.

filechooserbutton: Don't leak priv->model

applicationwindw: Don't leak show-help-overlay action

widget-factory: Don't leak GPropertyAction instances

build: Don't turn off critical warnings

-Wint-conversion is important because it checks casts from ints to
pointers.

-Wdiscarded-qualifiers is important to catch cases where we don't
strings when we should.

gtk-demo: Remove old build file

The includes are now generated with a Python script, so remvoe the old
perl script.

build: Remove old Makefile.inc files

They seem to have been forgotten in the meson transition

Update Polish translation

Some updates

Widget: make sure windowed widgets have a 1×1 min size

widget: Allow 0×0 allocations

Don't MAX the allocations with 1 anymore.

searchbar: Remove superfluous padding

So it's not 6px high when the child revealer is concealed.

box: Drop priv pointer

image: Remove mention on gdk-pixbuf-csource

gdk-pixbuf-csource as well as gdk_pixbuf_new_from_inline are deprecated.
Mention GResource instead.

HC: Add border/hilight when hovering flat buttons

The border and icon highlight are useful feedback that was defeated by
CSS precedence. It worked for .titlebuttons due to their implementation,
but the same was not true for custom .flat buttons. This makes it so.

https://bugzilla.gnome.org/show_bug.cgi?id=788580

HC: Flat titlebuttons became non-flat in backdrop

When the window was backdropped, they suddenly regained their border.
This was clearly not intentional or of any practical use to anyone.

Shuffle around some selectors so that the backdrop ones do not override
the flat ones and make the borders magically reappear when backdropped.

Note that, whereas standard titlebuttons get the border on :hover, other
.flat buttons in the headerbar do not. That should probably be fixed too

https://bugzilla.gnome.org/show_bug.cgi?id=788580

HighContrastInverse: Make window borders visible

They were hard-coded to a transparent black, but that is our bg colour
in HC Inverse, so windows stacked on top of each other or a dark
background blended together into a mush.

Fix this by making the $_wm_border* colours relative to the fg colour,
so that HighContrastInverse gets borders that are transparentised white.

https://bugzilla.gnome.org/show_bug.cgi?id=788575

HC: Fix MessageDialog’s rounded bottom borders

A missing decoration selector meant that we got a solid black background
behind the rounded corners of the dialog.

Copy the equivalent code from Adwaita, including nicely rounding the
focus outline too (& sorry, but this needs more newlines to be readable)

HighContrast: Fix titlebar containing >1 headerbar

by migrating the relevant code from Adwaita, to dodge unwanted doubling
up of the bottom border and such.

https://bugzilla.gnome.org/show_bug.cgi?id=769877

HighContrast: set min-height for headerbars

https://bugzilla.gnome.org/show_bug.cgi?id=784562

HC: Copy working tooltip code from Adwaita & tweak

There were various problems, like only selecting on .tooltip and not the
widget node tooltip, not being specific enough for tooltip.csd, etc. So,
specific theming was absent, and default popup window styles got applied

This commit copies in the better working tooltip CSS from Adwaita, but
applies a couple of changes to make it work better in the HC themes:
 • Reduce the transparency of the tooltip, so we achieve higher contrast
 • Drop the black text-shadow, as it is not useful on this more black bg

Note: we may then need to re-add some of this to the .tooltip class. But
it is unclear what needs done there. While Adwaita is not doing it, we
are better not to confuse by keeping it in HC only; we should try to be
as close as possible, to make it easier for HC to keep up with Adwaita.

https://bugzilla.gnome.org/show_bug.cgi?id=769879

HC: Give tooltips a border for contrast on Inverse

https://bugzilla.gnome.org/show_bug.cgi?id=769879

HighContrast: Actually style tooltips...

We need to select on the widget itself too, not just other things with
the .tooltip class.

https://bugzilla.gnome.org/show_bug.cgi?id=769879

[l10n] Updated Catalan (Valencian) translation

switch: Fix int/double comparison

We get a double coordinate.

scrolledwindow: Make sure gtk_get_current_event_state returned a state

Technically correct and pleases Coverity.

appchooserdialog: Remove unused macro

combobox: Remove unneeded queue_resize call

The remove/add/set_parent/unparent calls in there already do the
necessary work.

HighContrast: Correctly select window decoration

We need

    .window-classes decoration

but within the decoration parent selector, we were doing
&.window-classes, which gave us

    decoration.window classes

We need to fix this by selecting on .window-classes &

https://bugzilla.gnome.org/show_bug.cgi?id=788496

wayland: fix a typo

This was noticed in the gtk3 backport of these changes.

updateiconcache: Avoid confusing loop construct

n_attach_points is the result of g_strv_length(): the index at which the
string vector ends in NULL. So by definition, when i == n_attach_points,
string[i] == NULL, and there is no need to check for the latter. The
fact that we did appears to confuse static analysers, as the dereference
and index check were inverted from what would normally be safe. We could
reverse them, but we may as well just remove the unnecessary NULL check.

https://bugzilla.gnome.org/show_bug.cgi?id=788458

Enter submenus when activating their parent item

This gives consistent behavior with e.g. Qt, Mozilla's suites and
LibreOffice (with non-truly native backends like "gen" and "gtk",
but unlike "gtk2" and "gtk3" ones that probably use true GTK menus).

This behavior is expected by at least some accessibility users, and
it seems good to behave like other common applications and toolkits
in this area.  There should be no issue in doing so either for current
users, as it only enters the submenu instead of not doing anything.

https://bugzilla.gnome.org/show_bug.cgi?id=778811

-wayland: Safeguard against NULL gtk_surface1

There is no guarantee that the gtk_surface won't be NULL,
and Wayland API does not safeguard against NULL, so we have
to do that ourselves here.

We were also mistakenly cheking for the surface version off
by one, fix that too by checking if the surface version is
equal or greater.

TreeMenu: Remove accidentally duplicated doc lines

Not that it actually matters, as this is not public anymore, but this
was done accidentally. Also, sync a property description to ComboBox.

testcombo: Test grid :row- and :column-span-column

Ditch two items that were white and so weren’t visible on our standard
theme anyway, and use the new space to test extra grid-mode properties.

Note that if we do this then, as before, we set the ListStore on the
ComboBox before appending to it, that produced runtime warnings like:

Gtk-CRITICAL **: gtk_menu_attach: assertion 'left_attach < right_attach' failed

I didn’t look into that yet, but it may indicate that attaching items
vs. recognising their spans don’t occur in the correct order. For the
purposes of testing this, I just create the CB after filling its model.

ComboBox|TreeMenu: Slightly clarify wrap-width doc

ComboBox|TreeMenu: Clarify :row|column-span-column

ComboBox and TreeMenu warned in the doc for :row-span-column that the
value must not exceed :wrap-width, but :wrap-width does not interact
with the number of rows; it’s the :column-span-column that’s relevant.

Also: Warn that spans must be > 0 for rows too, and that column spans <=
:wrap-width are also not useful for items at menu column positions > 0.

Finally, refer to items having spans, not values, as we were already
talking about values in the model (and rows in the menu).

build: Bump minimum GLib version to 2.53.7

Otherwise it fails to build with:

GLib-Genmarshal-WARNING **: failed to open "--pragma-once":
No such file or directory

https://bugzilla.gnome.org/show_bug.cgi?id=788230

inspector: Add gsk and vulkan information

Show the used gsk renderer, as well as some information
about Vulkan, in the General tab.

widget-factory: Demo different Frame border-styles

This shows not all, but the interesting/less predictable styles, and
the ones that correspond to the :shadow-types that were here before.

Revert "widget-factory: Usefully demo Frame shadow types"

This reverts commit 6ee2bf628614e9c14b061bd48c2efd2c9515ab19.

There is a way to get different kinds of borders: it's CSS. It's better
to keep the 4 Frames and demo the different styles we can do using CSS.

Frame: Enhance CSS docs

Indicate how the old shadow types are replaced by CSS border-style et
al., and quote the names of nodes and classes for easier readability.

imcontext: Fix typo on ctrl+shift+u hex sequences handling

This condition was meant to exit when the event has no keyval (eg. fed a
non-keyboard event somehow).

snapshot: Fix mixup when creating crossfade nodes

We need to set the progress on the end node.

snapshot: Use one GPtrArray for all nodes

Instead of creating one GPtrArray per GtkSnapshotState and saving nodes
in there, create one GPtrArray per snapshot and assign a
start_node_index to every GtkSnapshotState as well as a n_nodes variable
so every state knows which nodes belong to it.

snapshot: Use one large GArray for all states

Allocating all of them separately shows up in profiles.

gtkroundedbox: Remove some unneeded conditionals

Always pass exactly 4 GtkCssValue* parameters to apply_border_radius
and make sure they are properly const.

Start reftests for gsk

Add a framework similar to what we have for gtk reftests
that lets us drop in a function to generate render nodes,
and compare reference output.

Create tests for gsk box shadows

Create tests for GskTextNode

Clip intermediate textures

In some cases, we were creating gigantic intermediate textures
only to clip out a small section afterwards (e.g. in the listbox
example in gtk4-demo). This is wasteful if we apply effects on
the texture, such as blur or color-matrix. So, clip the dimensions
of the intermediate texture with the current clip. To make this
feasible, we move the texture coordinate computation out of the
pipeline setup functions into the node_as_texture function where
this clipping happens.

One extra complication we encounter is that the node might get
clipped away completely. Since Vulkan does not allow to create
empty images, we bail out in this case and not draw anything.

With these changes, the listbox example in gtk4-demo goes from
32M pixels of intermediate texture to 320000.

stylecontext: Drop priv pointer

spinner: Remove priv pointer

switch: Remove priv pointer

label: Remove priv pointer

image: Remove some unneeded includes

image: Remove priv pointer

Look at all the free ram!

iconhelper: Don't inherit from GObject

It's not a GtkCssGadget anymore, it doesn't have any properties or
signals either and it's not public. Further, its lifetime is very clear
the way it's being used inside GTK+.

cssshadowsvalue: Don't heap-allocate shadows

This showed up in profiles in certain scenarios, so export a
_get_n_shadows getter instead and let callers provide a sufficiently
large allocated array of GskShadows, which we can use with
g_alloc/g_newa.

switch: Remove some unneeded includes

Add a forgotten rename

When I renamed blend to texture, I overlooked that the color
text pipeline also uses this shader. Fix it.

Reshuffle things a little bit

Instead of having a function with lots of arguments in
GskVulkanRender that we call from GskVulkanRenderPass which
then just calls back into GskVulkanRenderPass, just create
the new render pass object locally, and an api to add it
to the list that GskVulkanRender keeps. This makes it
a lot easier to preserve all the relevant parameters from
the parent render pass.

Small simplification

Just keep the viewport as a graphene_rect_t, we don't need
to use the Vulkan type here.

vulkan: Add a counter for intermediate textures

Count how many pixels of intermediate textures we create.
If we can count it, we can optimize it.

Cosmetics

Factor out a helper function.

Fix a doc comment

It was missing a return.

inspector: GSK_IS_RENDER_NODE() does NULL-check

That means we don't need an extra one just so gcc can complain.

Support repeat nodes in vulkan

We can use the texture pipeline for this, with a repeating sampler.

vulkan: Add a repeating sampler

None of the render ops use it, yet.

vulkan: Move sampler creation down

This is in preparation of allowing multiple samplers.

vulkan: Rename blend pipeline to texture

This is what it does, and the name blend is easily
confused with blend-mode.

vulkan: Allow to control scaling in the blend pipeline

Pass in a second rect that defines the size of the texture.
Update the sole caller to pass in the same rect, for unchanged
behavior.

vulkan: Fix setup for dependent render passes

We were not correctly preserving the modelview transform.

docs: css-properties: Fix wrongly repeated colour

testsuite: Add test case for previous commit

This previously led to a loop in the child widgets.

widget: Fix inserting child widgets at the beginning

We have to care about the child widget's current prev/next sibling as
well.

build: Add widgetorder unit test to build system

box: Remove return value from box_pack

Unused.

box: Remove size_allocate_no_center

GtkBox never has a center widget these days.

adwaita: Remove outlines from focused entries for now

Recorder: Add details for inset and outset shadow nodes

gsk: Fix a typo

Repeating linear gradient nodes share most of the implementation
with linear gradient nodes, but they shouldn't share the class name.

Recorder: Show details about linear gradient nodes

Recorder: Show details of blur nodes

recorder: Show details for blend nodes

gsk: remove gsk_blend_node_get_blend_mode from public api

We don't have any other such getters in the public api at
this point, so leave this one out as well.

Recorder: Show details of crossfade nodes

recorder: Show details for opacity nodes

Recorder: Show details of border nodes

recorder: Put text color in the right row

It ended up in the wrong place by accident.

Recorder: Show colors visually for text too

Just because we can do it.

Reshuffle the recorder ui a bit

Make it possible to see the properties and the rendering at
the same time, it makes navigating easier.

recorder: Show surfaces and textures

This makes the properties list much more interesting.

recorder: Show text node properties

recorder: show color information for color nodes

A first step towards showing more information for render nodes.

Plug some memory leaks

Make sure the scroll controllers in GtkComboBox and GtkScrolledWindow
get destroyed.